Skip to content

fix long idenitfier display issue#4866

Draft
anju15bharti wants to merge 8 commits into
babelfish-for-postgresql:BABEL_6_X_DEVfrom
amazon-aurora:BABEL-5975
Draft

fix long idenitfier display issue#4866
anju15bharti wants to merge 8 commits into
babelfish-for-postgresql:BABEL_6_X_DEVfrom
amazon-aurora:BABEL-5975

Conversation

@anju15bharti

@anju15bharti anju15bharti commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Description

[Describe what this change achieves - Guidelines below (please delete the guidelines after writing the PR description)]

  1. What is the change? This is best described in terms of “Currently, Babelfish does X. With this change it now does Y.” Think of “What did it used to do?” and “What does it do now?”
  1. Why was the change made? What drove our desire to put effort into the change?
  2. How was the code changed should only appear for large commits. This can serve as a rough roadmap to what’s contained in the commit. It should be very high level; if it’s directly referencing code it’s probably too detailed. It’s also critical that this section of a commit message does not try to replace proper code documentation (ie, block comments or README files). Generally, this section should only appear if the commit itself is large enough that it’s helpful to provide a roadmap to someone looking at the commit.
  3. The last descriptive piece is the “title” for the commit: the very first line of the commit message, which should typically be less than 80 characters. A good title is critical, because it’s the only thing that shows up in places like the Github commit listing. No one’s got time to read through full commit messages when trying to find a single commit out of dozens.

Engine : babelfish-for-postgresql/postgresql_modified_for_babelfish#773

Issues Resolved

[List any issues this PR will resolve]

Test Scenarios Covered

  • Use case based -

  • Boundary conditions -

  • Arbitrary inputs -

  • Negative test cases -

  • Minor version upgrade tests -

  • Major version upgrade tests -

  • Performance tests -

  • Tooling impact -

  • Client tests -

Check List

  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.

For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@anju15bharti anju15bharti force-pushed the BABEL-5975 branch 8 times, most recently from 457f277 to 0b27d70 Compare June 24, 2026 08:26
- Display original (non-truncated) names in all system views
- Rewrite truncated identifiers in TDS error messages
- Resolve original names from babelfish_identifier_mapping, reloptions, and attoptions
- Fix sp_statistics negative substring error
- Fix bbf_get_view_column_name crash on non-view relations
- Fix get_db_id to handle long database names
- Fix sp_helpdb to accept long database names
- Fix construct_unique_index_name buffer overflow
- Add sys.bbf_cur_db() for internal database_name comparisons
- Add comprehensive tests for all system objects with long identifiers
- Update upgrade script with view/procedure recreations

Signed-off-by: anju15bharti <abanju@amazon.com>
@anju15bharti anju15bharti force-pushed the BABEL-5975 branch 10 times, most recently from 7cdc4d2 to f89b57c Compare June 25, 2026 15:56
@anju15bharti anju15bharti force-pushed the BABEL-5975 branch 2 times, most recently from 9b8b073 to 9dec606 Compare June 26, 2026 08:27
@anju15bharti anju15bharti reopened this Jun 26, 2026
@anju15bharti anju15bharti force-pushed the BABEL-5975 branch 2 times, most recently from 2b8b963 to 154827f Compare June 26, 2026 11:55
@anju15bharti anju15bharti reopened this Jun 26, 2026
@anju15bharti anju15bharti force-pushed the BABEL-5975 branch 2 times, most recently from 25c5eca to 0899828 Compare June 27, 2026 06:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant